Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[12.0][IMP] Crea método para obtener si una factura se enviará como simplificada o completa #1409

Merged
merged 1 commit into from
Jul 4, 2020

Conversation

JuanjoA
Copy link
Contributor

@JuanjoA JuanjoA commented Jul 1, 2020

Al hilo de: #1171
La idea de este PR es facilitar la modificación de la forma en que se obtiene si una factura es simplificada o no.
Se veis el link anterior, hay casos en los que es mejor controlar a nivel de factura si cuando se envía al SII se hace como simplificada o no, y no a nivel de partner.
Este cambio permitiría heredar el método y cambiar la lógica para decidir si una factura es simplificada o no, permitiendo heredarlo y no tener que modificar sustancialmente los métodos en los que se comprueba el partner.sii_simplified_invoice

Entiendo que esto no afecta a la operativa actual, pero si mejora la integración con otras casuísticas.

A ver qué os parece.

Un saludo.

@OCA-git-bot
Copy link
Contributor

Hi @pedrobaeza,
some modules you are maintaining are being modified, check this out!

@pedrobaeza pedrobaeza added this to the 12.0 milestone Jul 1, 2020
@pedrobaeza
Copy link
Member

Me parece bien, pero haz squash de los commits en uno y no es necesario que cambies la versión del módulo. Ya lo hará el bot al fusionar y así se evitan posibles conflictos.

Copy link
Member

@pedrobaeza pedrobaeza left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Gracias por los tests extra

@pedrobaeza
Copy link
Member

@jalzaga vas a poder revisar el PR?

@pedrobaeza
Copy link
Member

Debes hacer rebase y no pull

@JuanjoA
Copy link
Contributor Author

JuanjoA commented Jul 3, 2020

Lo había corregido desde la interfaz de github, será por eso.

@pedrobaeza
Copy link
Member

Uf, por favor, huye de esa opción, porque estropea todo. Cógelo en local y haz squash.

Copy link
Contributor

@HaraldPanten HaraldPanten left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM a la espera de squash

@JuanjoA
Copy link
Contributor Author

JuanjoA commented Jul 4, 2020

¿Puede estar el travis cascado? Aquí se ve sin completar, pero al entrar indica todo ok.

@pedrobaeza
Copy link
Member

Sí, alguna vez le pasa. De todas formas, el bot lanza otro CI:

/ocabot merge patch

@OCA-git-bot
Copy link
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 12.0-ocabot-merge-pr-1409-by-pedrobaeza-bump-patch, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 81ff544 into OCA:12.0 Jul 4, 2020
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at 89ae496. Thanks a lot for contributing to OCA. ❤️

@OCA-git-bot OCA-git-bot changed the title [12.0][IMP] Crea método para obtener si una factura se enviará como simplificada o completa [12.0][IMP] Crea método para obtener si una factura se enviará como simplificada o completa Jul 4, 2020
@JuanjoA
Copy link
Contributor Author

JuanjoA commented Jul 4, 2020

Sí, alguna vez le pasa. De todas formas, el bot lanza otro CI:

/ocabot merge patch

Perfect, gracias.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants